singleselection: Fix a memory leak

3.98.1

a11y: Don't re turn uninitialized memory

For some reason, this was now showing up in the
a11y tests as random image sizes in the about
dialog test.

Merge branch 'filechooser-popover-fixes' into 'master'

Filechooser popover fixes

See merge request GNOME/gtk!1553

file chooser: Fix the rename file popover

Make this popover work again by attaching it
to the file chooser itself.

filechooser: Fix the file list popover

Make this popover work again.

filechooser: Fix the new folder popover

We were listening for change notification on a
no-longer-existing property.

places sidebar: Fix the popovers

We need to check-resize the popovers.

Merge branch 'portal-dir-chooser' into 'master'

filechoosernativeportal: Allowing selecting folder(s)

See merge request GNOME/gtk!1552

filechoosernativeportal: Allowing selecting folder(s)

Use the "directory" portal option when asked to select folders.

Merge branch 'portal-chooser-fail' into 'master'

portal file chooser: Handle errors

Closes #2527

See merge request GNOME/gtk!1551

portal file chooser: Handle errors

Fall back to showing a local file chooser
if we get DBus errors from the portal.

Closes: #2527

Merge branch 'fix-inspector-picking' into 'master'

inspector: Make picking work again

Closes #2528

See merge request GNOME/gtk!1549

Merge branch 'fix-inspector-key' into 'master'

x11: Fix key event state translation

See merge request GNOME/gtk!1550

x11: Fix key event state translation

In the event refactoring, a bit was lost.

This makes Ctrl-Shift-i work again to open
the inspector.

inspector: Make picking work again

We used to either make the inspector see- and click-thru
or lower it, but sadly translucency broke after we
removed surface opacity, and lowering doesn't work
on Wayland. So just hide the inspector window while
we are picking.

Fixes: #2528

Merge branch 'new-text-attributes' into 'master'

New text attributes

See merge request GNOME/gtk!1547

Make overline support conditional

The ci image does not have Pango 1.45, so for the
time being, make the overline support conditional
on Pango being new enough.

Require pango 1.44.4

This lets use use the insert-hyphen attribute
unconditionally.

gtk-demo: Add hyphenation control to the markup demo

gtk-demo: Allow word breaking in the markup demo

We want to show hyphenation control here.

textbuffer: Parse overline and hyphenation attributes

When inserting Pango markup into a text buffer, translate
Pango attributes for overlines and hyphenation control
into the corresponding text tag properties.

textlayout: Handle new text tag properties

Translate tag properties for overlines and
hyphenation control into Pango attributes.

texttag: Support overline and hyphenation control

Add support for recently added Pango attributes
for overlines and hyphenation control. The new
properties of GtkTextTag are
overline, overline-rgba, allow-breaks, show-spaces
and insert-hyphens.

Merge branch 'master_MR1290' into 'master'

Copy of !1290 for master

Closes #2365

See merge request GNOME/gtk!1546

Merge branch 'wip/chergert/fix-class-private-data-usage' into 'master'

Fix GtkWidgetClassPrivate data usage

See merge request GNOME/gtk!1545

widget: fix class private data usage to be _init() safe

Before this commit, adding GtkWidgetAction to class private data would
require copying the actions to each subclass as they were built or
modified. This was convenient in that it is a sort of "copy on write"
semantic.

However, due to the way that GTypeInstance works with base _init()
functions, the "g_class" pointer in GTypeInstance is updated as each
_init() function is called. That means you cannot access the subclasses
class private data, but only the parent class private data.

If instead we use a singly linked list of GtkWidgetAction, each subclass
has their own "head" yet all subclasses share the tail of the
GtkWidgetAction chain.

This creates one bit of complexity though. You need a stable way to know
which "bit" is the "enabled" bit of the action so we can track enabled
GAction state. That is easily solved by calculating the distance to the
end of the chain for a given action so that base classes sort ahead of
subclasses. Since the parent class always knows its parent's actions, the
position is stable.

A new dynamic bitarray helper also helps us avoid allocations in all the
current cases (up to 64 actions per widget) and dynamically switches to
malloc if that is to ever be exceeded.

widget: move class private data to gtkwidgetprivate.h

We want access to the private data from the action muxer so we can just
move the structures to the gtkwidgetprivate.h header. Alternatively we
could create accessors, but given that we'll probably need to use this
in other areas, seems reasonable to just put it there.

Merge branch 'test-protocol' into 'master'

Test protocol: TAP

See merge request GNOME/gtk!1543

testsuite: disable performance tests

For some reason, these tests are flaky in ci,
they always work locally for me. So, until
we use the data these tests produce for something,
lets just turn them off.

Merge branch 'wip/carlosg/text-handle-natives' into 'master'

Make text handles use native surfaces

See merge request GNOME/gtk!1487

gkimmulticontext: Fix crash due to leaked signal handler

The problem is caused by gtk_im_multicontext_set_slave(), which forgets
to disconnect these signal handlers:
* gtk_im_multicontext_retrieve_surrounding_cb
* gtk_im_multicontext_delete_surrounding_cb

If slave GtkImContext emits signal after GtkIMMulticontext context is
destroyed, this leads to reading freed memory, sometimes causing a crash.

Fixes: #2365

Merge branch 'ci-dbus-launch' into 'master'

ci: Add dbus-launch to the Docker build

See merge request GNOME/gtk!1544

testsuite: Fix an a11y test

The derive test wasn't producing TAP, so
running it as part of a tap testsuite wasn't
going well. Fix that.

testsuite: Tell meson we use TAP

This makes meson actually parse the individual test
results. Most of the time, it does not make a difference,
but one case where it does is when all the individual
tests of a binary are skipped, meson will mark the
test as skipped.

reftests: Fix an xfail to work

The background-image-multiple.ref.ui file uses
non-existing properties, which gives us a g_warning,
and the glib test framework insists on treating
warnings as fatal, so we end up doing exit(133),
which in turn makes the meson TAP parser ignore
its xfails.

Comment out the nonexisting properties, so we can
fail properly, and then in turn xfail properly.

testsuite: Don't exit unsuccessfully when using TAP

The meson TAP parser doesn't take this lightly and
forgets all about xfails when we exit(1), so don't.

ci: Add dbus-launch to the Docker build

gtktextview: Set magnifier popover on top

At the bottom, it sometimes has to fight for the same position than
text handles, besides might not be ergonomically convenient (eg.
finger/hand partly covering the popover). Move it at the top to fix
both.

gtktext: Set magnifier popover on top

At the bottom, it sometimes has to fight for the same position than
text handles, besides might not be ergonomically convenient (eg.
finger/hand partly covering the popover). Move it at the top to fix
both.

gtkpopover: Re-present surface if position/pointing-to change while visible

This ensures the popover will follow the new position parameters. This is
necessary for popovers like the text magnifier that can be repositioned
while visible.

gtkpopover: Ensure the pointed to rectangle has a minimum w/h

On wayland, protocol errors will be raised if the rectangle width/height
are 0. Avoid that situation so it's valid to all popovers.

gtkwindow: Remove internal popover API

This was only used by text handles, and not anymore.

gtktexthandle: Refactor and use native surfaces

Instead of being a GObject managing two GtkWidgets, make GtkTextHandle
a GtkWidget subclass, representing a single handle.

From the perspective of users (GtkText and GtkTextView), this is not a
big leap since they have to be aware of a great deal of text handles'
state. It actually makes things more direct and simple.

With text handles being widgets, those can be actual children of the
widget, and may have their own GdkSurface that we move around at will.
This is the second major aspect of this refactor.

reftests: Don't run tests with --verbose

Unexpected output tends to break TAP.

gsk: Don't unconditionally spew to stdout

testsuite: Don't g_print from tests

This breaks the TAP output.

testsuite: Fix the accessible test

This test was happily doing nothing.

Merge branch 'disable-window-test-2' into 'master'

Disable window test 2

See merge request GNOME/gtk!1541

temporarily disable window test

This test was broken by the GdkToplevel refactoring,
and needs a big gtkwindow.c configure request cleanup
before it will work again.

Revert "Merge branch 'disable-window-test' into 'master'"

This reverts commit 3ac4c76b18cc89a841ce09f0943539f16988fd21, reversing
changes made to 6ec96d2e989d029a303b8b20ec72b86f974c0e87.

Merge branch 'disable-window-test' into 'master'

Disable window test

See merge request GNOME/gtk!1540

Fix the action test

We have a test that enumerates the GtkText actions,
so when a new open appears, the test needs to be updated.

Fix a compiler warning

temporarily disable window test

This test was broken by the GdkToplevel refactoring,
and needs a big gtkwindow.c configure request cleanup
before it will work again.

text view: Fix touch selection

We forgot to allocated that popover.

widget: Drop the ::popup-menu signal

This is now done in widgets which have context
menus.

placesview: Stop using ::popup-menu

This signal is going away.

filechooser: Stop using ::popup-menu

This signal is going away.

mountoperation: Stop using ::popup-menu

We can just use a shortcut controller directly.

gtk-demo: Stop emitting ::popup-menu

The signal was not used anyway, in the font explorer demo.

colorchooser: Stop using ::popup-menu

This signal is going away. Use an action instead.

range: Remove ::popup-menu emission

This signal is going away, and having context menus
on sliders is not really a thing anyway.

scrollbar: Remove :popup-menu forwarding

This signal is going away.

emojichooser: Stop using ::popup-menu

This signal is going away. Use an action instead.

Use an action for the context menu keybinding

The ::popup-menu signal is going away.

Print mnemonic triggers clearly

inspector: Show shortcuts

At a tab that lists the shortcuts contained in a
GtkShortcutController.

Only create a class shortcut controller if we have shortcuts

No point in creating objects that just hold empty lists.

widget: Name the controllers

window: Name the the controllers

shortcutmanager: Name the controllers

This is helpful in the inspector.

Move shortcut manager initialization code

It is just too ugly to use quarks across multiple
source files, so add a private helper function that
attaches the controllers.

filechooser: Trigger the location popup via bindings

Simplifies code quite a bit.

accels: Remove GtkAccelGroup

testmenubutton: Don't create a GtkAccelGroup

It's unused.

testsuite: Remove GtkAccelGroup usage

widget: Remove gtk_widget_add_accelerator()

People should use shortcut controllers instead (global, capture).

A side effect of this is that GtkAccelLabel now lost its method to
magically look up accelerators to display. Somebody needs to add that
back later.

doc tools: Create AccelLabel image without using GtkAccelGroup

testgtk: Port keyval example to shortcut controllers

shortcutcontroller: Implement GtkBuildable

Use it to allow adding shortcuts to the controller via the usual <child>
method.

shortcutaction: INtegrate with GtkBuilder property parsing

<property name="action">action(win.quit)</property> style action
specifications now work for GtkShortcutAction properties.

shortcuttrigger: Add gtk_shortcut_triger_new_parse_string()

And hook it up into the GtkBuilder infrastructure.

application: Replace accelerator handling with shortcuts

API remains the same, but activation is now done via a
shortcutcontroller.

The code uses a controller with global scope so that the
shortcuts are managed with all the other global shortcuts.

accel: Add display arg to gtk_accelerator_parse_with_keycode()

It was using the default display unconditionally.

accelerators: Make gtk_accelerator_parse() return TRUE/FALSE

A parse function should return success or not. So do that.

shortcuttrigger: Add hash(), equal(), and compare() functions

Those are useful for putting triggers in hash tables or getting sorted
output.

shortcuts: Mananage managed shortcuts with a custom model

Reduce the amount of special casing by using a list model
for global and managed shortcuts, too.

This way, the ListModel API will work for the ShortcutController in the
GtkShortcutManager and GtkRoot.

The only special case remaining is shortcut activation, which needs to
pass the right widget to the controller in the global/managed case.

shortcutaction: Add gtk_shortcut_action_to_string()

For all but the callback action, we can print something useful.

widget: Keep keybindings as a GListStore

This way, we can use shortcut_controller_new_for_model() and avoid all
the special casing about run_class.

shortcutcontroller: Add gtk_shortcut_controller_new_for_model()

This is mainly for internal use, but I can't see a reason to not have it
public for people who want to maintain their own lists.

I'm sure gnome-builder will never ever find a way to misuse it.

shortcutcontroller: Implement GListModel

After all, this controller is a list of shortcuts.

gtk-demo: Port the sliding puzzle demo to shortcuts

shortcut: Change the API for creating shortcuts

When creating shortcuts, there almost always are a trigger and an action
available for use. So make gtk_shortcut_new() take those as arguments.

Also add gtk_shortcut_new_with_arguments() so people can easily pass
those in, too.

shortcut: Add GtkShortcutAction

Similar to GtkShortcutTrigger, GtkShortCutAction provides all the
different ways to activate a shortcut.

So far, these different ways are supported:

 - do nothing
 - Call a user-provided callback
 - Call gtk_widget_activate()
 - Call gtk_widget_mnemonic_activate()
 - Emit an action signal
 - Activate an action from the widget's action muxer
 - Activate a GAction

accelgroup: Remove unneeded APIs

After the removal of GtkAccelMap, these things are no longer necessary.

gtk: Remove GtkAccelMap

Now that accel paths are gone, the object managing them isn't needed
anymore either.

gtk: Remove accel paths

It's an outdated technology now that everybody is using GActionGroups.

If somebody wanted to support changeable shortcuts, they'd need to
reintroduce it in another way.

Remove GtkMnemonicHash

It's not used anymore.

window: Remove all old mnemonic handling API